Skip to content

test: add unit tests for lana vscode extensions#721

Merged
lcottercertinia merged 1 commit intocertinia:mainfrom
lukecotter:chore-lana-tests
Feb 9, 2026
Merged

test: add unit tests for lana vscode extensions#721
lcottercertinia merged 1 commit intocertinia:mainfrom
lukecotter:chore-lana-tests

Conversation

@lukecotter
Copy link
Copy Markdown
Contributor

📝 PR Overview

test: add unit tests for lana vscode extensions

We are missing test coverage so let's get it added!

  • Implement comprehensive tests for the SwitchTimelineTheme command, covering theme selection, sorting, and preview functionality.
  • Create tests for RawLogFoldingProvider to validate folding range extraction from log files, including edge cases and nested events.
  • Enhance ApexVisitor tests to handle null and undefined cases, ensuring robustness in method and class declarations.

🧩 Type of change (check all applicable)

  • 🐛 Bug fix - something not working as expected
  • ✨ New feature – adds new functionality
  • ♻️ Refactor - internal changes with no user impact
  • ⚡ Performance Improvement
  • 📝 Documentation - README or documentation site changes
  • 🔧 Chore - dev tooling, CI, config
  • 💥 Breaking change

📷 Screenshots / gifs / video [optional]

Show off your UI changes.

🔗 Related Issues

fixes #
resolves #206
closes #
related #

✅ Tests added?

  • 👍 yes
  • 🙅 no, not needed
  • 🙋 no, I need help

📚 Docs updated?

  • 🔖 README.md
  • 🔖 CHANGELOG.md
  • 📖 help site
  • 🙅 not needed

Anything else we need to know? [optional]

- Implement comprehensive tests for the SwitchTimelineTheme command, covering theme selection, sorting, and preview functionality.
- Create tests for RawLogFoldingProvider to validate folding range extraction from log files, including edge cases and nested events.
- Enhance ApexVisitor tests to handle null and undefined cases, ensuring robustness in method and class declarations.
@lcottercertinia lcottercertinia merged commit 9bf91db into certinia:main Feb 9, 2026
5 checks passed
@lukecotter lukecotter deleted the chore-lana-tests branch February 9, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: add missing tests for vscode features

2 participants